Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bug in configure-continuous-deployments command and maintain developer CLI #684

Conversation

tjementum
Copy link
Member

@tjementum tjementum commented Jan 26, 2025

Summary & Motivation

Fix an issue where the configure-continuous-deployments command failed when the az CLI threw an exception. Previously, these exceptions were handled correctly, but a recent update to how the CLI executes commands introduced an issue where errors were not always caught properly. This is now fixed.

Additionally, Azure region identifiers have been improved. The previous region abbreviations (e.g., gwc for Germany West Central) were not always intuitive. These have been updated to use country-specific ISO codes where applicable (e.g., us, ca, uk), and regional codes for areas with shared data sovereignty rules (e.g., eu for the European Union, as for Southeast Asia).

Other general maintenance updates have also been made to the developer CLI:

  • Ensure consistency in CLI descriptions and output messages by standardizing trailing periods.
  • Upgrade all NuGet packages, .NET SDK, and JetBrains tools to their latest versions.
  • Fix static code analysis warnings from SonarQube and JetBrains.
  • Remove ConfigurationSettings.ConfigurationSetting, which has been marked as obsolete for a while.
  • Add an explicit reference to System.IO.Pipelines Version=9.0.1 to avoid having to double build when the CLI detects changes.

Checklist

  • I have added tests, or done manual regression tests
  • I have updated the documentation, if necessary

@tjementum tjementum added Bug Something isn't working Enhancement New feature or request labels Jan 26, 2025
@tjementum tjementum self-assigned this Jan 26, 2025
@tjementum tjementum merged commit 611b601 into main Jan 26, 2025
16 checks passed
@tjementum tjementum deleted the pp-317-the-configure-continuous-deployments-cli-command-fails-when branch January 26, 2025 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant